Skip to content

Add CUD REST API endpoints for Roles#14840

Merged
ephraimbuddy merged 5 commits into
apache:masterfrom
astronomer:add-cud-endpoint-for-role-and-permission
Apr 5, 2021
Merged

Add CUD REST API endpoints for Roles#14840
ephraimbuddy merged 5 commits into
apache:masterfrom
astronomer:add-cud-endpoint-for-role-and-permission

Conversation

@ephraimbuddy

Copy link
Copy Markdown
Contributor

Closes: #14830
Add create, update and delete endpoints for Roles


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

@boring-cyborg boring-cyborg Bot added the area:API Airflow's REST/HTTP API label Mar 16, 2021
@ephraimbuddy ephraimbuddy added the AIP-38 Modern Web Application label Mar 22, 2021
Comment thread airflow/api_connexion/endpoints/role_and_permission_endpoint.py Outdated
Comment thread airflow/api_connexion/endpoints/role_and_permission_endpoint.py Outdated
Comment thread airflow/security/permissions.py Outdated
Comment thread tests/api_connexion/endpoints/test_role_and_permission_endpoint.py Outdated
@github-actions

Copy link
Copy Markdown
Contributor

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@ephraimbuddy ephraimbuddy force-pushed the add-cud-endpoint-for-role-and-permission branch from a430e7d to 62a0f0f Compare March 22, 2021 14:51
@ephraimbuddy ephraimbuddy requested review from ashb March 22, 2021 14:54
@ephraimbuddy ephraimbuddy force-pushed the add-cud-endpoint-for-role-and-permission branch from 62a0f0f to f74582c Compare March 29, 2021 18:09
@github-actions github-actions Bot added the full tests needed We need to run full set of tests for this PR to merge label Mar 31, 2021
@github-actions

Copy link
Copy Markdown
Contributor

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@ephraimbuddy ephraimbuddy force-pushed the add-cud-endpoint-for-role-and-permission branch from f74582c to ac1a194 Compare April 3, 2021 08:29
@ephraimbuddy ephraimbuddy reopened this Apr 4, 2021
@ephraimbuddy ephraimbuddy merged commit 6706b67 into apache:master Apr 5, 2021
@ephraimbuddy ephraimbuddy deleted the add-cud-endpoint-for-role-and-permission branch April 5, 2021 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AIP-38 Modern Web Application area:API Airflow's REST/HTTP API full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Create/Update/Delete API endpoints for Roles

3 participants